Skip to content

Migrate all CRs underneath CR heading#534

Merged
hugares merged 4 commits intokonflux-ci:mainfrom
enkeefe00:migrate-crs-in-docs-issue-533
Jan 19, 2026
Merged

Migrate all CRs underneath CR heading#534
hugares merged 4 commits intokonflux-ci:mainfrom
enkeefe00:migrate-crs-in-docs-issue-533

Conversation

@enkeefe00
Copy link
Copy Markdown
Contributor

@enkeefe00 enkeefe00 commented Dec 5, 2025

Moved the appstudio API custom resources under the "Custom Resources" heading, added links to the API documentation for each resource, and updated the API docs with npm.

Issue #533
Jira link: KFLUXINFRA-2700

@enkeefe00 enkeefe00 requested review from a team as code owners December 5, 2025 19:15
@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Dec 5, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request successfully reorganizes the Custom Resource documentation by consolidating all CRs under a single heading and adding direct links to their API references. The changes improve the structure and navigability of the documentation. I've identified a few minor typos that should be corrected to maintain documentation quality.

Comment thread hack/gen-api-docs.sh Outdated
Comment thread modules/ROOT/pages/getting-started.adoc Outdated
Comment thread modules/ROOT/pages/getting-started.adoc Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 5, 2025

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch 2 times, most recently from c85ec73 to 59f0418 Compare December 5, 2025 19:18
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 5, 2025

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

1 similar comment
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 5, 2025

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch from 59f0418 to 8fd82b3 Compare December 5, 2025 19:35
@enkeefe00 enkeefe00 requested a review from a team as a code owner December 5, 2025 19:35
@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch 2 times, most recently from 3af210b to 1123326 Compare December 5, 2025 19:42
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 5, 2025

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

@enkeefe00
Copy link
Copy Markdown
Contributor Author

enkeefe00 commented Dec 5, 2025

Comment thread hack/gen-api-docs.sh Outdated
@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch from 1123326 to 0428487 Compare December 16, 2025 17:17
@github-actions
Copy link
Copy Markdown

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

@enkeefe00 enkeefe00 requested a review from dirgim December 16, 2025 20:48
@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch from 0428487 to ab724d7 Compare January 15, 2026 19:00
@github-actions
Copy link
Copy Markdown

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch 2 times, most recently from 162fcac to 3e43ab9 Compare January 15, 2026 19:09
@github-actions
Copy link
Copy Markdown

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

@enkeefe00
Copy link
Copy Markdown
Contributor Author

@konflux-ci/build-maintainers can I get a review here. I'm trying to merge all of our doc PRs before the end of the sprint.

Copy link
Copy Markdown
Member

@dirgim dirgim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch from 9bcb372 to 3e43ab9 Compare January 19, 2026 16:25
Moved the appstudio API custom resources under the "Custom Resources"
heading, added links to the API documentation for each resource, and
updated the API dcos with npm.

konflux-ci#533
@enkeefe00 enkeefe00 force-pushed the migrate-crs-in-docs-issue-533 branch from 3e43ab9 to baf0e85 Compare January 19, 2026 16:29
@github-actions
Copy link
Copy Markdown

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

Comment thread modules/building/pages/creating.adoc
@github-actions
Copy link
Copy Markdown

🚀 Preview is available at: https://pr-534--konflux-docs.netlify.app

@enkeefe00 enkeefe00 requested a review from MartinBasti January 19, 2026 17:01
@hugares hugares merged commit 174eba4 into konflux-ci:main Jan 19, 2026
4 checks passed
@enkeefe00 enkeefe00 deleted the migrate-crs-in-docs-issue-533 branch January 19, 2026 18:07
@enkeefe00 enkeefe00 restored the migrate-crs-in-docs-issue-533 branch January 20, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants